Skip to content

Use npm and update bundling#3846

Merged
ntotten merged 3 commits intomainfrom
use_npm
Nov 28, 2025
Merged

Use npm and update bundling#3846
ntotten merged 3 commits intomainfrom
use_npm

Conversation

@ntotten
Copy link
Member

@ntotten ntotten commented Nov 28, 2025

No description provided.

Copilot AI review requested due to automatic review settings November 28, 2025 20:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the Prettier VS Code extension from pnpm to npm as the package manager and updates build/workflow configurations accordingly. The change simplifies the development setup by removing pnpm-specific workspace configurations while maintaining the same functionality.

Key Changes

  • Removed pnpm as the designated package manager (removed packageManager field and pnpm workspace files)
  • Updated all scripts and documentation from pnpm commands to npm run equivalents
  • Version bumped from 12.0.0-preview.1 to 12.0.0 (stable release)

Reviewed changes

Copilot reviewed 16 out of 19 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
package.json Removed packageManager field, updated version to 12.0.0, converted all pnpm script calls to npm equivalents
scripts/release.mjs Updated release script documentation and error messages to use npm run syntax
scripts/install-fixtures.mjs Changed default package manager from pnpm to npm, removed workspace-specific flags
pnpm-workspace.yaml Deleted pnpm workspace configuration file
pnpm-lock.yaml Deleted pnpm lockfile (npm lockfile not shown in diff)
lefthook.yml Updated git hooks to use npm run commands
docs/publishing.md Updated all release documentation examples to use npm run syntax
CONTRIBUTING.md Updated development documentation with npm commands, removed pnpm prerequisite
CLAUDE.md Updated AI context documentation with npm commands throughout

@ntotten ntotten merged commit ced800d into main Nov 28, 2025
10 checks passed
@ntotten ntotten deleted the use_npm branch November 28, 2025 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants